Skip to content

Conversation

@ilovepi
Copy link
Contributor

@ilovepi ilovepi commented May 6, 2025

I had forgotten to upload the formatting change.

I had forgotten to upload the formatting change.
Copy link
Contributor Author

ilovepi commented May 6, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@llvmbot
Copy link
Member

llvmbot commented May 6, 2025

@llvm/pr-subscribers-llvm-transforms

Author: Paul Kirth (ilovepi)

Changes

I had forgotten to upload the formatting change.


Full diff: https://github.com/llvm/llvm-project/pull/138620.diff

1 Files Affected:

  • (modified) llvm/lib/Transforms/Utils/Local.cpp (+1-2)
diff --git a/llvm/lib/Transforms/Utils/Local.cpp b/llvm/lib/Transforms/Utils/Local.cpp
index ce03bc066c6e2..1db01b2716a09 100644
--- a/llvm/lib/Transforms/Utils/Local.cpp
+++ b/llvm/lib/Transforms/Utils/Local.cpp
@@ -4225,9 +4225,8 @@ bool llvm::canReplaceOperandWithVariable(const Instruction *I, unsigned OpIdx) {
     return false;
 
   // Early exit.
-  if (!isa<Constant, InlineAsm>(I->getOperand(OpIdx))) {
+  if (!isa<Constant, InlineAsm>(I->getOperand(OpIdx)))
     return true;
-  }
 
   switch (I->getOpcode()) {
   default:

@ilovepi ilovepi merged commit 8404b29 into main May 6, 2025
8 of 12 checks passed
Copy link
Contributor Author

ilovepi commented May 6, 2025

Merge activity

@ilovepi ilovepi deleted the users/ilovepi/fix-bracing-local branch May 6, 2025 01:21
GeorgeARM pushed a commit to GeorgeARM/llvm-project that referenced this pull request May 7, 2025
I had forgotten to upload the formatting change.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants